-
Notifications
You must be signed in to change notification settings - Fork 26
Martinh/ntt support #463
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Martinh/ntt support #463
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some of these changes include changing Kaia back to Klaytn. But I think we should keep it as Kaia: https://x.com/klaytn_official?lang=en
I opened a PR along time ago to update the Wormhole SDK so it used Kaia instead, but I think it got lost in the sauce and I also tried updating the docs in that PR (probably unsuccessfully) so if I were them I probably just would have closed it 😆 - but I think it's still picking up dust somewhere 🌈
Maybe "Kaia (previously Klaytn)"?
Is Klaytn off? We are fetching updated data from this file, which Wormhole updates frequently |
That's what I'm trying to say, like I opened a PR against the SDK to update the names across the board so that the entire SDK would be using the correct name. But it didn't go anywhere. Regardless, Klaytn has official announcements saying they're now Kaia. So we should keep it as Kaia and if anything we can add (previously Klaytn) in parenthesis or something. But we shouldn't now be going backwards to Klaytn. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM Thanks!
Description
Update ntt supported tables using this file as reference.
Related to: papermoonio/wormhole-mkdocs#268
Checklist
wormhole-mkdocs
repo